Skip to content

feat(stacktrace): Wire SCM source context into new stack trace#111770

Merged
scttcper merged 5 commits intomasterfrom
scttcper/scm-source-context-new-stacktrace
Apr 7, 2026
Merged

feat(stacktrace): Wire SCM source context into new stack trace#111770
scttcper merged 5 commits intomasterfrom
scttcper/scm-source-context-new-stacktrace

Conversation

@scttcper
Copy link
Copy Markdown
Member

@scttcper scttcper commented Mar 28, 2026

same as #110327 but for the new stack trace. Fetches source content when a flag is enabled and the source is not available already.

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 28, 2026
@mujacica mujacica force-pushed the feat/scm-source-context-frontend branch 2 times, most recently from 68242f2 to da2b5bc Compare March 31, 2026 09:02
Base automatically changed from feat/scm-source-context-frontend to master April 2, 2026 08:23
scttcper and others added 2 commits April 2, 2026 15:12
Port on-demand SCM source context fetching from the old stack trace
component into the new component architecture, following the same
separation of concerns as line coverage:

- Issue layer (issueStackTrace/) checks the feature flag and passes
  hasScmSourceContext through StackTraceProvider context
- issueStackTraceFrameContext calls useSourceContext when frames lack
  embedded context, computing effectiveContext for FrameContent
- Generic layer (frame/) renders effective context and loading state
  via props without any feature flag awareness

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@scttcper scttcper force-pushed the scttcper/scm-source-context-new-stacktrace branch from b2652ae to 6628839 Compare April 2, 2026 22:13
@scttcper scttcper requested a review from mujacica April 2, 2026 22:30
@scttcper scttcper marked this pull request as ready for review April 2, 2026 22:30
@scttcper scttcper requested a review from a team as a code owner April 2, 2026 22:30
@scttcper scttcper changed the title feat(stacktrace): Wire SCM source context into new stack trace component feat(stacktrace): Wire SCM source context into new stack trace Apr 2, 2026
@scttcper scttcper merged commit fab31b9 into master Apr 7, 2026
67 checks passed
@scttcper scttcper deleted the scttcper/scm-source-context-new-stacktrace branch April 7, 2026 20:49
george-sentry pushed a commit that referenced this pull request Apr 9, 2026
same as #110327 but for the new
stack trace. Fetches source content when a flag is enabled and the
source is not available already.

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants